Skip to content

Add profile duplication SDK support - #863

Draft
leo-notte wants to merge 1 commit into
mainfrom
feat/profile-duplication
Draft

Add profile duplication SDK support#863
leo-notte wants to merge 1 commit into
mainfrom
feat/profile-duplication

Conversation

@leo-notte

@leo-notte leo-notte commented Jul 27, 2026 •
edited by blacksmith-sh Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

  • add ProfilesClient.duplicate
  • accept an optional destination name
  • send POST /profiles/{profile_id}/duplicate
  • cover endpoint construction and request payloads

Companion to nottelabs/monorepo#1918.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 0a38be48-4a3d-45ad-850d-f92cc760c2ed

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/profile-duplication

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Jul 27, 2026

Copy link
Copy Markdown

Greptile Summary

Adds profile duplication support to the Python SDK.

  • Adds ProfilesClient.duplicate using POST /profiles/{profile_id}/duplicate.
  • Adds typed request models for the optional destination name.
  • Adds unit coverage for endpoint construction and named request payloads.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The endpoint URL composes correctly under the profiles base path, the request type matches the public kwargs contract, optional null fields are omitted during serialization, and focused unit tests cover endpoint construction and the named payload.

Important Files Changed

Filename Overview
packages/notte-sdk/src/notte_sdk/endpoints/profiles.py Adds the typed profile-duplication endpoint and public client method consistently with existing SDK request patterns.
packages/notte-sdk/src/notte_sdk/types.py Adds synchronized Pydantic and TypedDict request types for the optional duplicate profile name.
tests/sdk/test_profile_endpoint_paths.py Verifies the duplicate endpoint method, relative path, response model, and named request payload.

Reviews (1): Last reviewed commit: "feat(sdk): add profile duplication" | Re-trigger Greptile

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown

Coverage

Tests Skipped Failures Errors Time
767 37 💤 2 ❌ 0 🔥 5m 51s ⏱️

@blacksmith-sh

blacksmith-sh Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Found 2 test failures on Blacksmith runners:

Failures

Test View Logs
pytest/test_does_nothing_expect_cleanup_personas View Logs
pytest/test_google_flights_with_agent View Logs

Fix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@leo-notte